Skip to content

feat: Inline tag creation in /inc lifecycle modals#244

Open
spalmurray wants to merge 10 commits into
mainfrom
spalmurray/RELENG-768
Open

feat: Inline tag creation in /inc lifecycle modals#244
spalmurray wants to merge 10 commits into
mainfrom
spalmurray/RELENG-768

Conversation

@spalmurray

Copy link
Copy Markdown
Contributor

Adds inline affected-service/region tag creation from the Slack lifecycle modals. The tag autocomplete offers a "+ Create " option (service/region only; impact_type stays curated) that get_or_creates the tag with the right type on submission, dedupes case-insensitively, handles uniqueness races, and degrades gracefully when the DB is unreachable. Closes RELENG-768.

@linear-code

linear-code Bot commented Jun 5, 2026

Copy link
Copy Markdown

RELENG-768

Comment thread src/firetower/slack_app/handlers/utils.py
Comment thread src/firetower/slack_app/tests/handlers/test_new_incident.py
Comment thread src/firetower/slack_app/handlers/new_incident.py
Comment thread src/firetower/slack_app/handlers/utils.py
@spalmurray spalmurray force-pushed the spalmurray/RELENG-768 branch from 5de96ba to 3ea47ac Compare June 11, 2026 20:03
@spalmurray spalmurray marked this pull request as ready for review June 11, 2026 20:04
@spalmurray spalmurray requested a review from a team as a code owner June 11, 2026 20:04

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 3ea47ac. Configure here.

Comment thread src/firetower/slack_app/handlers/new_incident.py
Comment thread src/firetower/incidents/models.py Outdated
Comment thread src/firetower/slack_app/handlers/mitigated.py Outdated
Comment thread src/firetower/slack_app/handlers/new_incident.py
Comment thread src/firetower/slack_app/tests/handlers/test_new_incident.py Outdated
Comment thread src/firetower/slack_app/handlers/resolved.py
"Could not resolve Slack user %s to a Firetower user",
form["captain_slack_id"],
# Work runs after ack() so the modal closes immediately; any failure is
# reported back to the submitter as an ephemeral message.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

concerned about hitting 3s ack timeout without this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants